apps/dashboard/app/(app)/apis/[apiId]/keys/[keyAuthId]/new/client.tsx (3)

Line range hint 449-451: Fix the refill day reset logic.

The comparison operator === is used instead of the assignment operator =, causing the refill day to not be reset properly when the interval is daily.

Apply this fix:

-    if (refill?.interval === "daily") {
-      refill?.refillDay === undefined;
-    }
+    if (refill?.interval === "daily") {
+      refill.refillDay = undefined;
+    }

---

Line range hint 308-318: Improve form reset logic for consistency.

The current implementation manually resets specific fields, which could lead to inconsistent form state. Consider using form.reset() to ensure all fields are properly reset to their default values.

Consider this approach:

-              onClick={() => {
-                key.reset();
-                form.setValue("expireEnabled", false);
-                form.setValue("ratelimitEnabled", false);
-                form.setValue("metaEnabled", false);
-                form.setValue("limitEnabled", false);
-                router.refresh();
-              }}
+              onClick={() => {
+                key.reset();
+                form.reset({
+                  prefix: defaultPrefix || undefined,
+                  bytes: defaultBytes || 16,
+                  expireEnabled: false,
+                  limitEnabled: false,
+                  metaEnabled: false,
+                  ratelimitEnabled: false,
+                });
+                router.refresh();
+              }}

---

Line range hint 779-789: Consider making the metadata textarea more responsive.

The textarea has a hardcoded height of 7 rows which might not be optimal for all screen sizes or content lengths. Consider making it more responsive or auto-resizing based on content.

Consider this enhancement:

                                      <Textarea
                                        disabled={!form.watch("metaEnabled")}
                                        className="m-4 mx-auto border rounded-md shadow-sm"
-                                       rows={7}
+                                       className="m-4 mx-auto border rounded-md shadow-sm min-h-[120px] max-h-[300px]"
+                                       rows={Math.min(10, Math.max(3, field.value?.split('\n').length || 3))}
                                        placeholder={`{"stripeCustomerId" : "cus_9s6XKzkNRiz8i3"}`}
                                        {...field}
                                        value={
                                          form.getValues("metaEnabled") ? field.value : undefined
                                        }
                                      />

packages/api/src/openapi.d.ts (1)

378-383: Consider enhancing the refillDay documentation.

While the documentation is good, it could be more explicit about:

• Valid day ranges (1-31)
• Behavior when specifying day 31 for months with fewer days
• Examples for different scenarios

Consider updating the documentation like this:

         /**
-         * @description The day verifications will refill each month, when interval is set to 'monthly'. Value is not zero-indexed making 1 the first day of the month. If left blank it will default to the first day of the month. When 'daily' is set for 'interval' 'refillDay' will be set to null.
+         * @description The day of the month (1-31) when verifications will refill, when interval is set to 'monthly'. 
+         *             - Value is not zero-indexed (1 = first day of month)
+         *             - Defaults to 1 if not specified
+         *             - For months with fewer days than specified (e.g. day 31 in February), refill occurs on the last day
+         *             - Set to null when interval is 'daily'
          * @default 1
          * @example 15
          */

Configuration used: CodeRabbit UI
Review profile: CHILL